Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Хмелёв Дима #51

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

NeSmogPridymatNick
Copy link

@NeSmogPridymatNick NeSmogPridymatNick commented Oct 31, 2016

@honest-hrundel honest-hrundel changed the title Хмелев Дмитрий Хмелёв Дима Oct 31, 2016
@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍏 Пройден линтинг и базовые тесты

@onufrienko
Copy link

Куда-то делись картинки :(
image

border: 5px double;
}

.headerOne

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

first-header

height: 75%;
}

.headerTwo

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

second-header

@onufrienko
Copy link

Не используй camelCase для именования классов

width: 15%;
}

.headerName

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Не очень удачное имя класса

font-family: 'Lobster', cursive;
}

.first

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Первое что?

width: 40%;
}

.second,

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Второе, третье, четвертое что? :)

@honest-hrundel
Copy link

🍏 Пройден линтинг и базовые тесты

.blackRect
{
background-color: black;
width: 20px !important;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Попробуй обойтись без !important

background-color: black;
}

.see

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Опять-таки не очень понятное имя

width: 100%;
}

.ogl

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ogl? :)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

черновой вариант , забыл исправить. Классов чет многа

width: 300px;
height: 216px;
border: 2px double;
float: right;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В этом задании нужно обойтись без плавающих элементов

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

А как тогда выравнивать картинки относительно блоков? Или вставлять картинку в середину текста?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Творить магию :)


.page
{
float: right;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

То же замечание, что и выше

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Либо что-то пошло не так, либо я все еще вижу float

font-size: 10px;
}

.pictureMainFirst

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

поясни, пожалуйста, название класса?

<title>Прибамбасск</title>
<meta charset="UTF-8">
<link href="index.css" rel="stylesheet">
<link href="https://fonts.googleapis.com/css?family=Lobster" rel="stylesheet">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Будет хорошо, если попробуешь подключить шрифты не только с google fonts, но и самостоятельно

<div class="world">
<img src="img/Мегаторг.png" class="mega" alt="Мегаторг" title="Магазин">
все товары<br>
<div class="big-world">Мира</div><br>
под одной крышей
</div>
</figure>
</div>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

почему ты переделал на div?

@onufrienko
Copy link

🍅

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍏 Пройден линтинг и базовые тесты

@@ -83,7 +83,7 @@
благодарность работникам порта. Среди главных
Факторов, определивших успех Прибамбасска, мэр
назвал удобное прибрежное расположение города —
рядом с Великим морем, а так же политику всячиски
рядом с Великим морем, а так же политику всячиски

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

всячески

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ахахахаха , я с картинки программой текст получал)

@onufrienko
Copy link

🍅

@honest-hrundel
Copy link

🍏 Пройден линтинг и базовые тесты

@onufrienko
Copy link

Почему я до сих пор вижу плавающий элемент и font-size small? :(

@honest-hrundel
Copy link

🍏 Пройден линтинг и базовые тесты

@onufrienko
Copy link

🚀

@forshtreter
Copy link

Тематически очень хорошо, реализация пока что не особо.

  • Шапка растягивается, содержимое – нет, выглядит странно. https://yadi.sk/i/F3WkZb47yQ9JC Надо растягивать содержимое тоже, тогда начинается самое интересное
  • Оглавление развалилось https://yadi.sk/i/jH9mh75zyQ9ay
  • Между баннерами и текстом какие-то странные дыры, между собой баннеры тоже как-то странно выровнены https://yadi.sk/i/aDNiCBXRyQ9ki

выпуск
</div>
<div class="header-name">
Вестник Прибамбасска<br>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Зачем этот br?

<img src="img/Море.png" alt="Море" title="Море">
</p>
<main>
<table>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Таблица тут вобще для раскладки не нужна.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ну тогда сложно padding-ми все выравнивать , так как float то же нельзя.Я первый раз все картинки флоатами выравнивал. Сказали нельзя. Я могу все inline дивами , все сделать. Но внутри все равно придется паддингами двигать

<h1>
Доходы растут
</h1>
<p class="main-picture">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

p – это paragraph, использовать его в качестве обёртки для картинки не лучшая идея, существует более специализированный тег именно для этой цели

<img src="img/Тотем.png" class="totem" alt="ТотемыРеклама" title="Реклама">
</td>
<td>
<div class="table-of-contents">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Оглавление можно сверстать как-нибудь поизящней, списком например

.header
{
position: relative;
overflow: hidden;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Зачем overflow: hidden?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

черновой вариант , забыл удалить


.first-header
{
height: 75%;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

75% чего?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

75% всего header

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Дело в том что у header-то высота не задана и поэтому это правило не работает.

@forshtreter
Copy link

forshtreter commented Nov 11, 2016

https://yadi.sk/i/eY2uheF4yQCxP этот блок такой странный, что я не могу понять, он так и должен выглядеть или развалился весь :)

Не знаю даже что посоветовать, давайте попробуем сделать хотя бы так, чтобы элементы в нём как-то более равномерно пространство занимали чтоли. Можно ещё купон выделить с помошью dashed бордера

@forshtreter
Copy link

То что надпись "ежедневно" прыгает туда-сюда в зависимости от ширины экрана это баг или фича? https://yadi.sk/i/Jk4snAvHyQDr4

@NeSmogPridymatNick
Copy link
Author

"этот блок такой странный, что я не могу понять, он так и должен выглядеть или развалился весь"
Да , так и должен выглядеть
http://natpagle.ru/wp-content/uploads/2016/10/vestnik-pribambasska-gazeta-02-0.jpg

@forshtreter
Copy link

Окей, этот вопрос снимается.

@forshtreter
Copy link

Но остальное в силе. Забыл помидор. 🍅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants